Skip to content

Fixing pytest decorator usage#217

Closed
MaximilianSoerenPollak wants to merge 7 commits intoeclipse-score:mainfrom
MaximilianSoerenPollak:MSP_fix_pytest_decorator
Closed

Fixing pytest decorator usage#217
MaximilianSoerenPollak wants to merge 7 commits intoeclipse-score:mainfrom
MaximilianSoerenPollak:MSP_fix_pytest_decorator

Conversation

@MaximilianSoerenPollak
Copy link
Contributor

@MaximilianSoerenPollak MaximilianSoerenPollak commented Jan 15, 2026 •
edited
Loading

Exchanging pytest marks to pytest decorator for XML attributes.

This PR needs this PR in docs as code to be merged & a new release of Docs-as-code first=> eclipse-score/docs-as-code#347

@github-actions
Copy link

github-actions bot commented Jan 15, 2026 •
edited
Loading

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ⚠️ Needs Review

Click to expand output 
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: f2042222-6c25-4d4d-aba3-7891e7c7ea16
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
DEBUG: Rule 'rust_qnx8_toolchain+' indicated that a canonical reproducible form can be obtained by modifying arguments integrity = "sha256-oEubHgeZDdT0svMmBKJx7c3/2TdSI/vfwRUyDn+TPGA="
DEBUG: Repository rust_qnx8_toolchain+ instantiated at:
  <builtin>: in <toplevel>
Repository rule http_archive defined at:
  /home/runner/.bazel/external/bazel_tools/tools/build_defs/repo/http.bzl:394:31: in <toplevel>
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'score_process', the root module requires module version score_process@1.3.2, but got score_process@1.4.0 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Computing main repo mapping: 
Loading: 
Loading: 3 packages loaded
Loading: 3 packages loaded
    currently loading: 
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)

Analyzing: target //:license-check (82 packages loaded, 10 targets configured)

Analyzing: target //:license-check (90 packages loaded, 13 targets configured)

Analyzing: target //:license-check (151 packages loaded, 2488 targets configured)

Analyzing: target //:license-check (159 packages loaded, 6997 targets configured)

Analyzing: target //:license-check (159 packages loaded, 6997 targets configured)

INFO: Analyzed target //:license-check (162 packages loaded, 9013 targets configured).
[13 / 14] [Prepa] Generating Dash formatted dependency file ...
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 62 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 23.879s, Critical Path: 0.47s
INFO: 14 processes: 5 disk cache hit, 9 internal.
INFO: Build completed successfully, 14 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@github-actions
Copy link

github-actions bot commented Jan 15, 2026

The created documentation from the pull request is available at: docu-html

@MaximilianSoerenPollak
Copy link
Contributor Author

MaximilianSoerenPollak commented Jan 15, 2026

This then should look like:

image

PiotrKorkus
PiotrKorkus reviewed Jan 16, 2026
View reviewed changes
tests/python_test_cases/tests/test_cit_default_values.py Outdated

import pytest
from testing_utils import LogContainer, ScenarioResult
from attribute_plugin import add_test_properties
Copy link
Contributor

@PiotrKorkus PiotrKorkus Jan 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add this file https://github.com/eclipse-score/reference_integration/blob/main/feature_integration_tests/test_cases/test_properties.py and decorator via it?
This will allow non-bazel execution

tests/python_test_cases/pytest.ini Outdated
env = D:RUST_BACKTRACE = 1
junit_family = xunit1
filterwarnings =
error
Copy link
Contributor

@PiotrKorkus PiotrKorkus Jan 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this one is too general and shouldnt be added

PiotrKorkus
PiotrKorkus reviewed Jan 16, 2026
View reviewed changes
@MaximilianSoerenPollak MaximilianSoerenPollak marked this pull request as ready for review January 19, 2026 06:00
@MaximilianSoerenPollak MaximilianSoerenPollak marked this pull request as draft January 19, 2026 06:00
@MaximilianSoerenPollak
Copy link
Contributor Author

MaximilianSoerenPollak commented Jan 21, 2026

#224 fixes the conflicts etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PiotrKorkus PiotrKorkus PiotrKorkus left review comments

@vinodreddy-g vinodreddy-g Awaiting requested review from vinodreddy-g vinodreddy-g will be requested when the pull request is marked ready for review vinodreddy-g is a code owner

@PandaeDo PandaeDo Awaiting requested review from PandaeDo PandaeDo will be requested when the pull request is marked ready for review PandaeDo is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@MaximilianSoerenPollak @PiotrKorkus